Is this an ok addition to be making here? Or does there need to be a WeakRef polyfill which falls back to some sort of a no-op class wrapper?

(in case it's useful, WeakRef was introduced in node v16.4: mdn link)

🟡 Missing processor metadata update in localTrackRepublished handler causes stale/empty metadata after reconnection

The PR adds setRoom(this) calls in localTrackPublished, localTrackUnpublished, trackSubscribed, and trackUnsubscribed handlers to push processor metadata. However, the localTrackRepublished handler (fired during full reconnect when the SDK auto-republishes tracks) has no corresponding setRoom call or metadata push. After republish, the publication receives a new SID via publication.updateInfo(newInfo) at room.ts:567, but the track's info.sid (used by pushProcessorMetadataToStream at track.ts:125) is never updated. This means any subsequent call to pushProcessorMetadataToStream (e.g., when a new AudioStream is registered on the track after reconnection) will fail the lookup publication.sid === trackSid at track.ts:141 because the publication now has the new SID while the track still has the old one. The processor receives empty participantIdentity and publicationSid.

(Refers to lines 566-570)

In the localTrackRepublished handler in room.ts (around line 561-573), after the publication info is updated and re-keyed in the map, the track's processors are never notified of the new publication SID. Two things need to happen:

1. The track's own info.sid needs to be updated to match the new publication SID, because pushProcessorMetadataToStream in track.ts uses this.sid (which reads this.info?.sid) to look up the publication in the trackPublications map.

2. A metadata push should be triggered so that processors attached to existing AudioStreams on this track receive the updated stream info (new publicationSid).

A fix would be to add something like this inside the `if (publication)` block in the localTrackRepublished handler, after the publication is re-keyed:

  if (publication.track) {
    publication.track.info = { ...publication.track.info, sid: publication.sid }; // or however TrackInfo is updated
    publication.track.setRoom(this); // re-pushes metadata to all registered audio streams
  }

Note: setRoom(this) when oldRoom === room will still properly re-register the listener and push metadata to streams. However, be careful that the track's info structure is updated correctly (it's a protobuf TrackInfo object).

Was this helpful? React with 👍 or 👎 to provide feedback.